hm I agree having good unit test coverage is important -- this seems too trivial to test? It is also part of the code that doesn't really change.

Hi @rxin,

Firstly, thanks for quick reply.

I was thinking for unit-test coverage perspective and a starter point to contribute project but it is ok for me if PR is closed as well ;)

This is to some degree covered by other tests, yes. It's probably just not worth 100 lines of test code to test the contents of a Set from all angles.

I think the other two non-test changes are trivial but I'd be willing to commit them.

Thanks @srowen for the comment. The change-set looks ready.

OK, can you modify the title? it becomes the commit message. "Minor code change to TaskState and Task" or something.

Normally I'd say this is too trivial to commit by itself but I know it didn't start that way, and because it's a first commit I'm more inclined to merge even a little part of it to get you through the process once.

Thanks @srowen. PR Title is updated.

Test build #3282 has finished for PR 15143 at commit da2da9f.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

Merged to master